projects
/
ostree.git
/ commitdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
| commitdiff |
tree
raw
|
patch
| inline |
side by side
(parent:
e3e82c5
)
libotutil: Plug a leak
author
Krzesimir Nowak
<krzesimir@kinvolk.io>
Tue, 24 May 2016 08:36:38 +0000
(10:36 +0200)
committer
Atomic Bot
<atomic-devel@projectatomic.io>
Tue, 24 May 2016 11:36:15 +0000
(11:36 +0000)
There's no need to allocate the variant builder on a heap, so allocate
it on the stack and avoid a memory leak at the same time.
Closes: #307
Approved by: cgwalters
src/libotutil/ot-variant-utils.c
patch
|
blob
|
history
diff --git
a/src/libotutil/ot-variant-utils.c
b/src/libotutil/ot-variant-utils.c
index 2dc07582c39065c4d29ab78058236199d592f55a..7b54542ee7ac1756400e9e1ac5d6696ad2ee259d 100644
(file)
--- a/
src/libotutil/ot-variant-utils.c
+++ b/
src/libotutil/ot-variant-utils.c
@@
-33,7
+33,10
@@
GVariant *
ot_gvariant_new_empty_string_dict (void)
{
- return g_variant_builder_end (g_variant_builder_new (G_VARIANT_TYPE ("a{sv}")));
+ g_auto(GVariantBuilder) builder = {{0,}};
+
+ g_variant_builder_init (&builder, G_VARIANT_TYPE ("a{sv}"));
+ return g_variant_builder_end (&builder);
}
GVariant *